Skip to content

refactor!: Refactor AlloyDBVectorStore and AlloyDBEngine to depend on PGVectorstore and PGEngine respectively #434

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Aug 13, 2025

Conversation

dishaprakash
Copy link
Contributor

@dishaprakash dishaprakash commented Jul 23, 2025

refactor!: Refactor AlloyDBVectorStore and AlloyDBEngine to depend on PGVectorstore and PGEngine respectively

AlloyDBAsyncVectorStore and AlloyDBVectorStore are refactored to depend on the AsyncPGVectorStore and PGVectorStore respectively as well as AlloyDBEngine to depend on PGEngine.

Hybrid search is now supported in AlloyDBVectorStore. Tests for hybrid search have been added.

VectorIndexes also are directly linked to the ones in LangChain Postgres

BREAKING CHANGE:
The support for text filters in similiarity searches has been removed, and we support only dictionary filters now.

The dependency on langchain-core and pgvector is based on LangChain-Postgres's specifications.

@dishaprakash dishaprakash requested review from a team as code owners July 23, 2025 08:31
@product-auto-label product-auto-label bot added the api: alloydb Issues related to the googleapis/langchain-google-alloydb-pg-python API. label Jul 23, 2025
@dishaprakash dishaprakash changed the title chore: Refactor AlloyDBEngine to depend on PGEngine refactor!: Refactor AlloyDBEngine to depend on PGEngine Jul 24, 2025
@dishaprakash dishaprakash changed the title refactor!: Refactor AlloyDBEngine to depend on PGEngine refactor: Refactor AlloyDBEngine to depend on PGEngine Aug 5, 2025
averikitsch
averikitsch previously approved these changes Aug 8, 2025
* refactor!: Refactor AlloyDBVectorStore to depend on PGVectorstore

* Linter fix

* Fix tests

* Fix tests

* fix tests

* linter fix

* fix vectorstore

* add all existing tests

* fix tests

* fix tests

* fix tests

* fix tests

* fix tests

* fix tests

* fix tests

* fix tests

* fix dependecies

* Add tests for hybrid search

* linter fix

* fix test

* Add better documentation for breaking change

* minor fix

* update deps

* Re-expose hybrid search config

* add header

* linter fix
averikitsch
averikitsch previously approved these changes Aug 12, 2025
@dishaprakash dishaprakash changed the title refactor: Refactor AlloyDBEngine to depend on PGEngine refactor!: Refactor AlloyDBVectorStore and AlloyDBEngine to depend on PGVectorstore and PGEngine respectively Aug 13, 2025
@averikitsch averikitsch merged commit 2293bd7 into main Aug 13, 2025
13 checks passed
@averikitsch averikitsch deleted the refactor-engine branch August 13, 2025 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: alloydb Issues related to the googleapis/langchain-google-alloydb-pg-python API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants